-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Антонов Алексей #24
base: master
Are you sure you want to change the base?
Антонов Алексей #24
Conversation
🍏 Пройден линтинг и базовые тесты |
🍏 Пройден линтинг и базовые тесты |
1 similar comment
🍏 Пройден линтинг и базовые тесты |
index.css
Outdated
.h-view:checked ~ .main .cat | ||
{ | ||
display: block; | ||
width: 1000px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Когда показывается списком, то если разрешение экрана у пользователя меньше 1000px, он будет расстроен
index.css
Outdated
margin: 15px 0; | ||
} | ||
|
||
.h-view:checked ~ .main .img-content |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай переименуем h-view
и v-view
на более понятные названия
index.css
Outdated
|
||
.h-view:checked ~ .main .description | ||
{ | ||
width: 400px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай сделаем верстку поадаптивнее, например, можно посмотреть в сторону min-width
или max-width
<h1 class="logo">Мяндекс.<span>Муррркет</span></h1> | ||
<p> | ||
<label for="vertical"> | ||
<img src="images/v-view.jpg" alt="Горизонтальные плитки" title="Горизонтальные плитки"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Молодец, что сделал не обычным инпутом 👍
Привет. Мне очень понравилась твоя работа) Есть пара некритичных замечаний, но я надеюсь, что ты их поправишь 😉 Еще есть последний вопрос: будешь ли делать звездочки? |
🚀 |
🍏 Пройден линтинг и базовые тесты |
index.html
Outdated
<main class="main"> | ||
<div class="cat male"> | ||
<div class="img-content"> | ||
<img src="images/01.jpg" alt="" title="Генрих 4"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Пустой alt
|
||
.price:after | ||
{ | ||
content: '₽'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В случае с ценой так делать, пожалуй, не стоит, потому что текст в псевдоэлементе нельзя выделить и скопировать. Но в целом пододное внимание к мелочам – это очень хорошо 👍 Так что можно не исправлять :)
index.html
Outdated
</p> | ||
</header> | ||
<main class="main"> | ||
<div class="cat male"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Алексей, многовато div'ов. Здесь, например, подойдёт article, для обёртки картинки figure, для имени кота – заголовок.
🍅 |
Пока помидор, но в целом очень даже неплохо, по звёздочке вобще никаких вопросов, только мелочёвку подправить. |
🍅 Не пройден линтинг или базовые тесты |
🍏 Пройден линтинг и базовые тесты |
🍏 Пройден линтинг и базовые тесты |
Посмотреть решение